lib/view/page/home_tab.dart (1)

32-36: 💤 Low value

Snippet tab uses the same icon for both selected and unselected states.

Unlike other tabs which use distinct icons (e.g., bx_server/bxs_server, terminal_outlined/terminal, folder_open/folder), the snippet tab uses Icons.code for both states. Consider using Icons.code_outlined or similar for the unselected state to maintain visual consistency.

♻️ Suggested fix

       AppTab.snippet => NavigationDestination(
-        icon: const Icon(Icons.code),
+        icon: const Icon(Icons.code_outlined),
         label: libL10n.snippet,
         selectedIcon: const Icon(Icons.code),
       ),

Apply the same change to navRailDestination at lines 57-61.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@lib/view/page/home_tab.dart` around lines 32 - 36, AppTab.snippet's
NavigationDestination uses the same Icon(Icons.code) for both selected and
unselected states; change the unselected icon to a distinct outlined variant
(e.g., Icons.code_outlined) in the NavigationDestination for AppTab.snippet and
make the same change in the corresponding navRailDestination block so the
unselected/selected icons mirror the pattern used by other tabs.

lib/view/page/home.dart (1)

163-170: 💤 Low value

Consider passing _onDestinationSelected directly.

The wrapper callback is unnecessary since _onDestinationSelected already matches the expected void Function(int) signature.

♻️ Suggested simplification

     if (Platform.isMacOS) {
       return PlatformMenuBar(
-        menus: MacOSMenuBarManager.buildMenuBar(context, (int index) {
-          _onDestinationSelected(index);
-        }),
+        menus: MacOSMenuBarManager.buildMenuBar(context, _onDestinationSelected),
         child: mainContent,
       );
     }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@lib/view/page/home.dart` around lines 163 - 170, The anonymous wrapper around
_onDestinationSelected is redundant; update the PlatformMenuBar call to pass the
function directly by replacing the inline closure in
MacOSMenuBarManager.buildMenuBar with _onDestinationSelected (which already
matches void Function(int)), leaving PlatformMenuBar and mainContent unchanged.